Skip to content

Fix Lambda Test Tool handling of missing start options - #2525

Merged
normj merged 1 commit into
aws:masterfrom
maoyouaa:mao/fix-test-tool-missing-start-options
Sep 17, 2026
Merged

normj merged 1 commit into
aws:masterfrom
maoyouaa:mao/fix-test-tool-missing-start-options

Conversation

@maoyouaa

Copy link
Copy Markdown
Contributor

Issue #, if available:

Fixes #2523

Description of changes:

  • Treat missing emulator or event source options as an expected user error instead of an unhandled exception.
  • Preserve the actionable parameter message without displaying the “This is a bug” banner.
  • Add regression coverage and a patch change file.

Testing:

  • .NET 8 unit tests: 197 passed, 1 skipped (198 total)
  • .NET 10 unit tests: 197 passed, 1 skipped (198 total)
  • Verified that start with no options exits with code 1 and displays only the parameter guidance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@normj

normj commented Sep 17, 2026

Copy link
Copy Markdown
Member

@maoyouaa were you planning in more changes on this PR before taking this out of draft? The change and purpose look pretty straight forward.

@maoyouaa

Copy link
Copy Markdown
Contributor Author

No further changes are planned. The patch and regression coverage are complete, so I'll mark this ready for review. Thanks for checking.

@maoyouaa
maoyouaa marked this pull request as ready for review September 17, 2026 22:15
@maoyouaa
maoyouaa requested review from a team as code owners September 17, 2026 22:15
@maoyouaa
maoyouaa requested a review from normj September 17, 2026 22:15
@normj

normj commented Sep 17, 2026

Copy link
Copy Markdown
Member

Thanks for PR @maoyouaa. The PR just missed going in today's release. So it will be included in the next one. I assume there is not an immediate rush on this PR being released.

@normj
normj merged commit 52b1966 into aws:master Sep 17, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

During start command got bug error

3 participants